Skip to content

Handle rearrivals#660

Open
labkey-martyp wants to merge 9 commits intorelease25.11-SNAPSHOTfrom
25.11_fb_nirc_rearrival
Open

Handle rearrivals#660
labkey-martyp wants to merge 9 commits intorelease25.11-SNAPSHOTfrom
25.11_fb_nirc_rearrival

Conversation

@labkey-martyp
Copy link
Contributor

Rationale

Some animals leave and come back. The EHR system doesn't really handle this. This is a work around to allow multiple arrivals and allow protocol/project assignments on animals with multiple arrivals. Some manual data manipulation still required.

Also a housing performedby fix.

Ticket 54440: Unable to have multiple arrivals
Ticket 54425: Housing Add From File Peformed By

Changes

  • Fix data type issue on performedby default values
  • Add rearrival form.
  • Update triggers to prevent reusing Id on arrival but not rearrival
  • Update assignment triggers

Copy link
Contributor

@labkey-bpatel labkey-bpatel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but please see my comments.

{
public NIRCRearrivalFormSection()
{
super("study", "arrival", "Rearrivals", "ehr-gridpanel", true, true, true);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should the query name here be 'rearrival'?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its still the arrival dataset, just a different form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants